View Issue Details

IDProjectCategoryView StatusLast Update
0001583T99X171.00 SKB EaglexTS certificationpublic2024-05-09 13:51
Reporter(SW) Jason LingAssigned To(SW) Kerwin Chen Due Date
PrioritynormalSeveritys4-minorReproducibilityalways
Status closedResolutionreopened 
Summary0001583: [Smart3][ATV10/12] Do NOT exclude LatinIMEGoogleTVPrebuilt from Firmware
DescriptionHi YK,

Our QA find that there is a modification violate Google Requirement

Please check the following requirement (Requirement.png & GTVS_list.png)

We found that SPTEK prebuilt the SKBIme and override LatinIMEGoogleTVPrebuilt (override.jpg)

Please forward it to SPTEK, and fix it on Eagle10/12, Swan10.

Thanks,
Jason

TagsNo tags attached.
Attach Tags

Users monitoring this issue

Activities

(SW) Jason Ling

2022-12-22 16:58

developer  

Requirement.png (8,141 bytes)   
Requirement.png (8,141 bytes)   
override.jpg (30,865 bytes)   
override.jpg (30,865 bytes)   
GTVS_list.png (90,458 bytes)   
GTVS_list.png (90,458 bytes)   

(ALTech) Younkwang Jung

2022-12-23 08:53

developer   ~0011552

Hi Jason

I reported it to SPTEK
https://jira.skbroadband.com/browse/FSTB12-24

Thank you
YK.Jung

(SW) Jason Ling

2023-01-16 17:29

developer   ~0011802

Hi YK,

I've seen the LatinIMEGoogleTVPrebuilt is already included in FW.

Thanks,
Jason

(ALTech) Younkwang Jung

2023-02-08 15:07

developer   ~0012066

 Hi Jason

There is not enough free space because LatinIMEGoogleTVPrebuilt is included in the FW.

INNOPIA requested 3PL to check if LatinIMEGoogleTVPrebuilt can be removed.
The answer for the 3PL is...
If there is no problem with Google Test, you can remove it.
So, INNOPIA is currently in a situation where LatinIMEGoogleTVPrebuilt has been removed from FW.

Can FXN also check with 3PL to see if it is possible to remove this apk file?

Thank you
YK.Jung

(SW) Jason Ling

2023-02-08 16:03

developer   ~0012067

Hi YK,

I think INNOPIA's 3PL doesn't read requirement carefully.

I've attache the requirement. Android TV devices running Android 12 "MUST" include all required Google apps, with the following APKs.

Even though automated test doesn't require test LatinIMEGoogleTVPrebuilt.
But we can not violate the rule.

I attache the ATV11's requirement.
ATV11's rules are more stricter than the ATV12's

Not only prebuilt LatinIMEGoogleTVPrebuilt but also set it as default IME.

Thanks,
Jason

(SW) Jason Ling

2023-02-08 16:03

developer   ~0012068

Android-11.jpg (260,188 bytes)   
Android-11.jpg (260,188 bytes)   

(ALTech) Younkwang Jung

2023-02-09 14:29

developer   ~0012088

Hi Jason

I got it ,
Thanks

YK.Jung

(SW) Jason Ling

2023-02-13 16:30

developer   ~0012150

Hi YK,

Did SPTEK plan to exclude LatinIMEGoogleTVPrebuilt from FW, because I saw the following commit.

commit db12b158f7e0f656a6374cd3a966ef225ef7d32f
Author: echwang <echwang@sptek.co.kr>
AuthorDate: Mon Jan 2 17:27:31 2023 +0900
Commit: echwang <echwang@sptek.co.kr>
CommitDate: Mon Jan 2 17:27:31 2023 +0900

    Revert "[FSTB12-24] [구글인증] LatinIMEGoogleTVPrebuilt 제거시 구글 requirement 위반 (ATV10/ATV12)"

    This reverts commit d1da9efe3f7647a527e58426c34e100ef599f19e.

Thanks,
Jason

(ALTech) Younkwang Jung

2023-02-14 08:30

developer   ~0012161

Hi Jason

First of all, for Smart3/Smart3mini/AI models, SKB/SPTEK decided to remove LatinIMEGoogleTVPrebuilt as default.
(Removed due to lack of free space), so the commit you mentioned was done ( it is a SPTEK common module )

Here, we will add LatinIMEGoogleTVPrebuilt only for FXN models ( Smart3 ATV10 / ATV12 , AISoundMAX ATV10)
So please make a Android.mk(or makefile) to add LatinIMEGoogleTVPrebuilt only for FXN models.
I think it is not difficult, but SKB requested it to us.
SPTEK will then apply it

please check it

Thank you
YK.Jung

(SW) Jason Ling

2023-02-14 08:53

developer   ~0012162

Hi YK,


>>Here, we will add LatinIMEGoogleTVPrebuilt only for FXN models
We have already added.

The question is when the vendor/skb/prebuilts/skb_ime/Android.mk was executed
The following line will replace LatinIMEGoogleTvPrebuilt with BTV_IME.
LOCAL_OVERRIDES_PACKAGES := LatinIME LatinIMEGoogleTvPrebuilt

This Android.mk belongs to vendor/skb/prebuilts/skb_ime/

Can we modify it ourselves?

Thanks,
Jason

(ALTech) Younkwang Jung

2023-02-14 10:16

developer   ~0012165

Hi Jason

I discussed it with SPTEK.
This ime folder is a common folder, so SPTEK will send me the user guide(Flag setting method) because it can cause issues with other models if FXN modify it incorrectly.
I think you can apply that method.

I will let you know when I receive the guide after SPTEK's test.

Thank you
YK.Jung

(ALTech) Younkwang Jung

2023-02-15 14:23

developer   ~0012177

HI Jason

FXN can decide whether to include LatinIMEGoogleTvPrebuilt by setting the flag below on device.mk.

Inlcude GoogleTV IME : INCLUDE_GOOGLETV_IME = true
Exclude GoogleTV IME : INCLUDE_GOOGLETV_IME = false or not exist (default)

SPTEK has committed as follows. ( please check attached piture )
LOCAL_OVERRIDES_PACKAGES := LatinIME
ifneq ($(INCLUDE_GOOGLETV_IME), true)
LOCAL_OVERRIDES_PACKAGES += LatinIMEGoogleTvPrebuilt
endif

If you want to add LatinIMEGoogleTvPrebuilt,
please add INCLUDE_GOOGLETV_IME = true for each model at device.mk and commit.

Thank you
YK.Jung
image.png (61,626 bytes)   
image.png (61,626 bytes)   

(ALTech) Younkwang Jung

2024-05-09 13:41

developer   ~0015738

Hi Jason

LatinIMEGoogleTvPrebuilt is included.
You can close this ticket

Thank you
YK.Jung

Issue History

Date Modified Username Field Change
2022-12-22 16:58 (SW) Jason Ling New Issue
2022-12-22 16:58 (SW) Jason Ling Status new => assigned
2022-12-22 16:58 (SW) Jason Ling Assigned To => (ALTech) Younkwang Jung
2022-12-22 16:58 (SW) Jason Ling File Added: Requirement.png
2022-12-22 16:58 (SW) Jason Ling File Added: override.jpg
2022-12-22 16:58 (SW) Jason Ling File Added: GTVS_list.png
2022-12-23 08:09 (ALTech) Younkwang Jung Issue Monitored: (ALTech) SY Yoon
2022-12-23 08:09 (ALTech) Younkwang Jung Issue Monitored: (ALTech) JunGyu Kim
2022-12-23 08:52 (ALTech) Younkwang Jung Issue Monitored: (ALTech) Sangmin Choi
2022-12-23 08:53 (ALTech) Younkwang Jung Note Added: 0011552
2023-01-16 17:29 (SW) Jason Ling Status assigned => resolved
2023-01-16 17:29 (SW) Jason Ling Resolution open => fixed
2023-01-16 17:29 (SW) Jason Ling Note Added: 0011802
2023-01-16 17:30 (SW) Jason Ling Status resolved => closed
2023-02-08 15:07 (ALTech) Younkwang Jung Assigned To (ALTech) Younkwang Jung => (SW) Jason Ling
2023-02-08 15:07 (ALTech) Younkwang Jung Status closed => feedback
2023-02-08 15:07 (ALTech) Younkwang Jung Resolution fixed => reopened
2023-02-08 15:07 (ALTech) Younkwang Jung Note Added: 0012066
2023-02-08 16:03 (SW) Jason Ling Category SW Issue => xTS certification
2023-02-08 16:03 (SW) Jason Ling Note Added: 0012067
2023-02-08 16:03 (SW) Jason Ling Note Added: 0012068
2023-02-08 16:03 (SW) Jason Ling File Added: Android-11.jpg
2023-02-08 16:04 (SW) Jason Ling Assigned To (SW) Jason Ling => (ALTech) Younkwang Jung
2023-02-09 14:29 (ALTech) Younkwang Jung Note Added: 0012088
2023-02-09 14:29 (ALTech) Younkwang Jung Status feedback => closed
2023-02-13 16:30 (SW) Jason Ling Status closed => feedback
2023-02-13 16:30 (SW) Jason Ling Note Added: 0012150
2023-02-14 08:30 (ALTech) Younkwang Jung Note Added: 0012161
2023-02-14 08:53 (SW) Jason Ling Note Added: 0012162
2023-02-14 08:53 (SW) Jason Ling Status feedback => assigned
2023-02-14 10:16 (ALTech) Younkwang Jung Note Added: 0012165
2023-02-15 14:23 (ALTech) Younkwang Jung Note Added: 0012177
2023-02-15 14:23 (ALTech) Younkwang Jung File Added: image.png
2024-05-09 13:41 (ALTech) Younkwang Jung Note Added: 0015738
2024-05-09 13:41 (ALTech) Younkwang Jung Assigned To (ALTech) Younkwang Jung => (SW) Jason Ling
2024-05-09 13:51 (SW) Kerwin Chen Assigned To (SW) Jason Ling => (SW) Kerwin Chen
2024-05-09 13:51 (SW) Kerwin Chen Status assigned => closed